-
-
Notifications
You must be signed in to change notification settings - Fork 165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove the checkboxes for showing correct answers and solutions. #2275
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
drgrice1
force-pushed
the
checkbox-removal
branch
4 times, most recently
from
December 14, 2023 13:34
cfcb1a5
to
4f26479
Compare
drgrice1
force-pushed
the
checkbox-removal
branch
3 times, most recently
from
December 19, 2023 13:53
fb95bf4
to
126af5f
Compare
drgrice1
force-pushed
the
checkbox-removal
branch
2 times, most recently
from
January 8, 2024 22:47
e2332b5
to
8718106
Compare
drgrice1
force-pushed
the
checkbox-removal
branch
from
January 16, 2024 14:43
8718106
to
7eee92d
Compare
pstaabp
approved these changes
Jan 24, 2024
drgrice1
force-pushed
the
checkbox-removal
branch
from
January 24, 2024 22:22
7eee92d
to
ab13a03
Compare
Solutions are just shown when they can be. This is the same as this is for homework problems. There is just no reason for the checkbox.
Correct answers are shown when they "can" be. However, there are two cases in which I don't think that it is appropriate to show the correct answers without any user interaction. This is where the corresponding PG pull request that adds a "Reveal" button for correct answers comes in. The first case is in "Show Me Another" problems when the course configuration is set to allow correct answers to be shown for these problems. Students should be able to see feedback messages and such without seeing the correct answer. The point of these problems is for students to obtain practice working problems without simply looking at the correct answers, so an additional step should be needed to see the correct answer. The second case is for instructors viewing problems before answers are available for students. The instructor may be viewing the problem with students present (for example, when working problems in class) and may want to be able to show messages and such without showing the correct answer. There is a new course configuration option in the "Optional Modules". If this option is true, then answer feedback is immediately available in problems after answers are available. Students do not even need to click "Submit Answers" to make this feedback appear. Furthermore, the $showPartialCorrectAnswers variable set in some problems that prevents showing which of the answers are correct is ignored if this option is true. This option is true by default, but if it is beleived that the majority won't want this, then it can be switched to false. This option was actually added as an afterthought. It was initially implemented with automatic feedback after answers are available with no option to disable that behavior. I would also be happy reverting to that if we feel that this is something everyone will want. This automatic feedback is particularly useful for tests that have multiple pages. Currently you need to change the page, click the show correct answers checkbox, then click check test, and that has to be repeated for each page. That is a real pain if each problem is on its own page. The check box is gone, but you still have to click check test after changing pages. With this option the feedback is there with each page change (the same as it was before when the problem grader is enabled). Also the %must hash has been removed. It was not used at all except in Problem.pm for one permission, and that usage was incorrect. It resulted in recordAnswers always being forced to be true with the default setting of avoid_recording_answers set to "nobody". The %must hash can not force an option to be disabled. It can only force it to be enabled. This permission was done correctly in the GatewayQuiz.pm module, so that approach is now used in Problem.pm also.
Now if the `$pg{options}{automaticAnswerFeedback}` option is true, then feedback will also be automatically shown anytime a student returns to a problem that has already been answered.
This option makes it so that correct answers are not visible and the "Reveal" button must be clicked to show them even after answers are available for students. This was requested by @dlglin.
drgrice1
force-pushed
the
checkbox-removal
branch
from
February 2, 2024 20:47
ab13a03
to
f866bdd
Compare
drdrew42
approved these changes
Feb 7, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove the "Show solutions" checkbox for tests (a.k.a. gateway quizzes) Solutions are just shown when they can be. This is the same as this is for homework problems. There is just no reason for the checkbox in tests either.
The show correct answers checkbox is also removed for both homework and tests. Correct answers are just shown when they "can" be. However, there are two cases in which I don't think that it is appropriate to show the correct answers without any user interaction. This is where the corresponding PG pull request (openwebwork/pg#982) that adds a "Reveal" button for correct answers comes in.
The first case is in "Show Me Another" problems when the course configuration is set to allow correct answers to be shown for these problems. Students should be able to see feedback messages and such without seeing the correct answer. The point of these problems is for students to obtain practice working problems without simply looking at the correct answers, so an additional step should be needed to see the correct answer.
The second case is for instructors viewing problems before answers are available for students. The instructor may be viewing the problem with students present (for example, when working problems in class) and may want to be able to show messages and such without showing the correct answer.
There is a new course configuration option in "Problem Display/Answer Checking". If this option is true, then answer feedback is immediately available in problems after answers are available. Students do not even need to click "Submit Answers" to make this feedback appear. Furthermore, the $showPartialCorrectAnswers variable set in some problems that prevents showing which of the answers are correct is ignored if this option is true. This option is true by default, but if it is believed that the majority won't want this, then it can be switched to false. This option was actually added as an afterthought. It was initially implemented with automatic feedback after answers are available with no option to disable that behavior. I would also be happy reverting to that if we feel that this is something everyone will want.
This automatic feedback is particularly useful for tests that have multiple pages. Currently you need to change the page, click the show correct answers checkbox, then click check test, and that has to be repeated for each page. That is a real pain if each problem is on its own page. The check box is gone, but you still have to click check test after changing pages. With this option the feedback is there with each page change (the same as it was before when the problem grader is enabled).
Also the %must hash has been removed. It was not used at all except in Problem.pm for one permission, and that usage was incorrect. It resulted in recordAnswers always being forced to be true with the default setting of avoid_recording_answers set to "nobody". The %must hash can not force an option to be disabled. It can only force it to be enabled. This permission was done correctly in the GatewayQuiz.pm module, so that approach is now used in Problem.pm also.
Edit: This now takes the automatic feedback option a bit further (as requested by @dlglin). Now the automatic feedback is shown anytime that a student returns to a problem that has been previously answered.
Also there is another new option added. If the
$pg{options}{correctRevealBtnAlways}
option is true, then the "Reveal" button will be shown instead of the correct answer being immediately visible anytime that correct answers are available in the feedback (except when the problem grader is active and the last time that a student sees a problem before needing to request a new problem version when problem randomization and show correct on randomize are enabled). This option is false by default. This was also requested by @dlglin.